Skip to content

Conversation

@baggepinnen
Copy link
Contributor

@baggepinnen baggepinnen commented Sep 25, 2024

This PR depends on:

Note to self: If one of the errors

 The initial condition for parameter body₊body₊m isn't provided.
car₊body₊m cannot be converted to Float64. Please provide a default value for the symbolic variables.

appear, it might be due to parameters being forwarded straight through a component like BodyShape using the kwargs..., and the default for the parameter in the inner body does not find the corresponding parameter in the ParentScope (BodyShape). By introducing the same parameter in the BodyShape, this is resolved.

It can also happen if pars aren't provided to the ODESystem constructor in the component that defines them.

@baggepinnen baggepinnen merged commit 714fc3d into main Oct 7, 2024
2 checks passed
@baggepinnen baggepinnen deleted the paramfixes branch October 7, 2024 14:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants